Skip to content

Conversation

dhurlburtusa
Copy link
Contributor

This change is to help the documentation to fall more inline with the v6.x Promise spec terminology.

Use "Fulfilled" in place of "resolved".
Use "Pending" in place of "loading" when "loading" didn't feel more natural.
Capitalize "O" in object when used as a datatype to use capitalization preferred by JSDoc and other tools. Only boolean, number, and string are expected to be all lowercase (assuming you are expecting the primitives and not the wrapper objects).
Fix a few minor mistakes (grammar, unescaped tag syntax, etc).

Use "Fulfilled" in place of "resolved".
Use "Pending" in place of "loading" when "loading" didn't feel more natural.
Capitalize "O" in object when used as a datatype to use capitalization preferred by JSDoc and other tools. Only `boolean`, `number`, and `string` are expected to be all lowercase (assuming you are expecting the primitives and not the wrapper objects).
Fix a few minor mistakes (grammar, unescaped tag syntax, etc).
@codecov
Copy link

codecov bot commented Mar 29, 2019

Codecov Report

Merging #38 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   99.61%   99.61%           
=======================================
  Files           5        5           
  Lines         525      525           
  Branches      112      112           
=======================================
  Hits          523      523           
  Misses          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23c9d8d...7d95c01. Read the comment docs.

Copy link
Member

@ghengeveld ghengeveld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent improvements, thanks for the sharp eye.

@ghengeveld ghengeveld merged commit 94ec322 into async-library:master Mar 29, 2019
@dhurlburtusa dhurlburtusa deleted the patch-1 branch March 29, 2019 21:45
@ghengeveld
Copy link
Member

@all-contributors please add @dhurlburtusa for doc

@allcontributors
Copy link
Contributor

@ghengeveld

I've put up a pull request to add @dhurlburtusa! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants